Skip to content

build: bump x/net + x/text to clear two HIGH/CRITICAL image CVEs#151

Closed
EdHasNoLife wants to merge 1 commit into
mainfrom
bump-x-net-x-text-cves
Closed

build: bump x/net + x/text to clear two HIGH/CRITICAL image CVEs#151
EdHasNoLife wants to merge 1 commit into
mainfrom
bump-x-net-x-text-cves

Conversation

@EdHasNoLife

Copy link
Copy Markdown
Collaborator

main is red on Image Build (Trivy: 2 HIGH/CRITICAL per image). The last merge was docs-only, so this is a vulnerability-DB update, not a code regression — two CVEs published against indirect deps already in the tree.

CVE package installed fixed
CVE-2026-46600 golang.org/x/net v0.55.0 0.56.0
CVE-2026-56852 golang.org/x/text v0.37.0 0.39.0
  • govulncheck stays green — neither path is reachable from our code. Trivy scans the image SBOM regardless of reachability, so the gate fires on presence alone.
  • Bumping the deps rather than narrowing the gate, per the standing rule that a red vulnerability gate is fixed by upgrading, not by continue-on-error.
  • go.mod/go.sum only; make pre-pr clean (no generated-code drift).

Unblocks every open PR — Image Build runs on all of them.

main is red on Image Build (Trivy) as of the last merge, and nothing in
that merge caused it -- it was docs-only. Two CVEs were published against
indirect deps already in the tree:

  CVE-2026-46600  golang.org/x/net  v0.55.0 -> 0.56.0
                  (panic parsing an invalid SVCB/HTTPS RR in dns/dnsmessage)
  CVE-2026-56852  golang.org/x/text v0.37.0 -> 0.39.0
                  (infinite loop on invalid input)

govulncheck stays green because neither path is reachable from our code,
but Trivy scans the image SBOM regardless of reachability -- so the gate
fires on presence. Bump the deps rather than narrow the gate, matching the
standing rule for govulncheck.

Blocks every open PR, since Image Build runs on each.
@github-actions

Copy link
Copy Markdown

Codex review

Files reviewed

Root:

  • go.mod — [reviewed]
  • go.sum — [reviewed]

Findings

Blocking

None.

Should-fix

None.

Nit

None.

Per-category coverage

  1. Vendor-neutral naming: Clean
  2. Contract & spec / docs sync: Clean
  3. gRPC fail-open semantics: Clean
  4. Quality (tests / error wrapping / package placement / generated drift): Clean

Verdict

approve

@EdHasNoLife

Copy link
Copy Markdown
Collaborator Author

Superseded by #150, which landed the identical bump (x/net v0.56.0, x/text v0.39.0) and is already merged to main. Closing this duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant